Skip to content

Conversation

valentinbujdoso
Copy link
Contributor

…date is true #242

@valentinbujdoso
Copy link
Contributor Author

Im not clearly understand how can I "fake" it. could you explaint it a little bite detailed?

@magwas
Copy link
Member

magwas commented Apr 4, 2019

Im not clearly understand how can I "fake" it. could you explaint it a little bite detailed?

https://blog.pragmatists.com/test-doubles-fakes-mocks-and-stubs-1a7491dfa3da

@valentinbujdoso
Copy link
Contributor Author

So something like this?

Copy link
Member

@magwas magwas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see poc/assurancemanager branch in my repo to understand how dependency injection (old school) works.
Build on that code.

@magwas
Copy link
Member

magwas commented Apr 9, 2019

Please provide full code coverage. Yes, even for the code I helped you with.

…e_obtained_from_a_cast_vote_if_canupdateis_true_242
@magwas
Copy link
Member

magwas commented Apr 11, 2019

only an unimplemented test is needed for ADAAssuranceManager.getAssurances()

List<String> getAssurances();

static AssuranceManager getAssuranceManager() {
return new ADAAssuranceManager();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test it to kill zombie.

@valentinbujdoso
Copy link
Contributor Author

I will take a look tomorrow

…e_obtained_from_a_cast_vote_if_canupdateis_true_242
@valentinbujdoso
Copy link
Contributor Author

@magwas I created the test but it give back quite strange errors

@magwas
Copy link
Member

magwas commented Apr 16, 2019

you have received a pull request yesterday with the resolution of the conflicts.

…_can_be_obtained_from_a_cast_vote_if_canupdateis_true_242_java10_merge

merge with current edemo/develop
…e_obtained_from_a_cast_vote_if_canupdateis_true_242
@valentinbujdoso
Copy link
Contributor Author

The import javax.xml.ws.WebServiceContext; cannot be resolved.
What should I do?

@magwas
Copy link
Member

magwas commented Apr 17, 2019

seems like jax-ws is missing. window/install new software.
Is it with the eclipse in the docker image or your own setup?

@szirbucz szirbucz mentioned this pull request Apr 29, 2019
@magwas
Copy link
Member

magwas commented Jun 4, 2019

its going in in #333 @valentinbujdoso is it paid yet for you?

@valentinbujdoso
Copy link
Contributor Author

@magwas:
Elnezest, hogy el vagyok veszve, a majusom nagyon suru volt, es a juniusom meg talan meg surubb lesz, de utana jelentkezem munkara keszen.
Ezt meg nem rendeztuk, ha visszatertem az elok korebe beszelunk rola.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants